-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Update Custom Indexes section in api.rst #10517
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
A couple of xarray-tutorial paths have changed.
Hmmm, yeah looks like some last minute changes for the scipy 2025 tutorial :) xarray-contrib/xarray-tutorial#334 |
doc/api.rst
Outdated
@@ -1571,12 +1571,22 @@ Custom Indexes | |||
============== | |||
.. currentmodule:: xarray | |||
|
|||
Default, pandas-backed indexes built-in to Xarray: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering about renaming this entire section Custom Indexes -> Indexes
, and promoting it up so that it's underneath other core data structures (under Coordinates
)
Also a quick update here would be nice https://xray--10517.org.readthedocs.build/en/10517/internals/internal-design.html#internal-design (but that can also happen later as part of #10323)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved half of it up, and left the "build new indexes" pieces down where they are now.
The internal design page will require more time than I have now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
A couple of xarray-tutorial paths have changed.